lib/checkout: fallback to checksum for UNION_IDENTICAL
authorJonathan Lebon <jlebon@redhat.com>
Tue, 10 Oct 2017 18:14:10 +0000 (18:14 +0000)
committerAtomic Bot <atomic-devel@projectatomic.io>
Sat, 14 Oct 2017 13:19:18 +0000 (13:19 +0000)
commit950318936286b1b1ffcc2894a846af447e8dab12
tree4c87dd29ae7a902233903d36cd78f104ac15ce44
parent2a9c5efe1d10b79681b0ee638994ead375be6597
lib/checkout: fallback to checksum for UNION_IDENTICAL

There's a subtle issue going on with the way we use `UNION_IDENTICAL`
now in rpm-ostree. Basically, the crux of the issue is that we checkout
the whole tree from the system repo, but then overlay packages by
checking out from the pkgcache repo. This is an easy way to break the
assumption that we will be merging hardlinks from the same repo.

This ends up causing issues like:
https://github.com/projectatomic/rpm-ostree/issues/1047

There, `vim-minimal` is already part of the host and has an object for
`/usr/share/man/man1/ex.1.gz`. `vim-common` has that same file, but
because it's unpacked in the pkgcache repo first, the hardlinks are not
the same.

There are a few ways we *could* work around this in rpm-ostree itself,
e.g. by re-establishing hardlinks when we do the content pull into the
system repo, but it still felt somewhat hacky. Let's just do this the
proper way and fall back to checksumming the target file if needed,
which is what librpm does as well in this case. Note that we only
checksum if they're not hard links, but they're the same size.

Closes: #1258
Approved by: cgwalters
src/libostree/ostree-core-private.h
src/libostree/ostree-core.c
src/libostree/ostree-repo-checkout.c
tests/basic-test.sh